Skip to content

Add missing setImmediates method#588

Merged
Kangz merged 2 commits into
webgpu-native:mainfrom
beaufortfrancois:fix-immediate
May 7, 2026
Merged

Add missing setImmediates method#588
Kangz merged 2 commits into
webgpu-native:mainfrom
beaufortfrancois:fix-immediate

Conversation

@beaufortfrancois

Copy link
Copy Markdown
Contributor

This PR adds missing the setImmediates method as described in gpuweb/gpuweb#5423

@beaufortfrancois

Copy link
Copy Markdown
Contributor Author

@kainino0x Please review

@beaufortfrancois

Copy link
Copy Markdown
Contributor Author

Hold on a bit, it looks like it should match dawn.json, not WebGPU Javascript API. I'll update it shortly. Sorry for the noise.

@kainino0x
kainino0x marked this pull request as draft April 29, 2026 14:43
@kainino0x

Copy link
Copy Markdown
Collaborator

No worries, marking as draft

@beaufortfrancois
beaufortfrancois marked this pull request as ready for review April 29, 2026 15:19
@beaufortfrancois

Copy link
Copy Markdown
Contributor Author

We should be good now!

@kainino0x kainino0x left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm! I think we'll hold on landing this until the JS spec is landed even though I think it's very unlikely to change at this level.

(marking draft again just to prevent landing)

@kainino0x
kainino0x marked this pull request as draft April 30, 2026 08:04
@Kangz

Kangz commented May 7, 2026

Copy link
Copy Markdown
Collaborator

The WebGPU spec PR landed, doing the same for this one.

@Kangz
Kangz marked this pull request as ready for review May 7, 2026 13:39
@Kangz
Kangz merged commit 673658b into webgpu-native:main May 7, 2026
5 checks passed
@beaufortfrancois
beaufortfrancois deleted the fix-immediate branch May 11, 2026 06:49
hmaarrfk added a commit to conda-forge-admin/webgpu-headers-feedstock that referenced this pull request Jul 21, 2026
<details><summary>Claude's draft</summary>

wgpu-native v29.0.1.1 bumped its ffi/webgpu-headers submodule from
7d3186c3dd2c708703524027b46b8703534ab3cc to
673658bc2bd70ec39fc55ebe6bb0173cf6d0a603 ("Add missing setImmediates
method", webgpu-native/webgpu-headers#588, 2026-05-07), so track that
commit here. Note v27.0.4.1 and v29.0.0.0 both still pointed at
7d3186c3, which is why this only moves now.

The header diff is small: adds the WGPUWGSLLanguageFeatureName_
LinearIndexing enum value and the wgpu{Compute,Render}PassEncoder
SetImmediates proc pointer declarations.

Verified that webgpu.h at 673658bc is byte-identical to the copy
vendored in wgpu-py 0.32.0 (wgpu/resources/webgpu.h), which is the
release that targets wgpu-native 29.0.1.1 -- so this is exactly the
right commit for that stack.

Built locally with .ci_support/linux_64_.yaml: produces
webgpu-headers-0.0.0.2026.5.7.673658b-h707e725_0.conda shipping
include/webgpu.h with the expected contents. The new version string
sorts above the current 0.0.0.2026.3.19.7d3186c per conda's
VersionOrder, so the build number stays at 0.

Resume this Claude session:
```
cd /Users/mark/git/feedstocks
claude --resume 373eb0f8-42bc-4c6c-aa5f-2136f7a0cc4f
```
</details>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants